Skip to content

elasticsearch: various test fixes#4299

Merged
josephwoodward merged 2 commits intomainfrom
mmt/elasticsearch-fixes-v2
May 4, 2026
Merged

elasticsearch: various test fixes#4299
josephwoodward merged 2 commits intomainfrom
mmt/elasticsearch-fixes-v2

Conversation

@mmatczuk
Copy link
Copy Markdown
Contributor

Commits

  • tolerate context.Canceled in v8 test stream goroutine
  • tolerate context.Canceled in v9 test stream goroutine

Jira

  • CON-438
  • CON-449

When StopWithin cancels the stream context, Run returns
context.Canceled. Match the mssqlserver fix so the goroutine does not
fail the test on expected shutdown.

Fixes CON-438
When StopWithin cancels the stream context, Run returns
context.Canceled. Match the v8 fix so the goroutine does not fail the
test on expected shutdown.

Fixes CON-449
@claude
Copy link
Copy Markdown

claude Bot commented Apr 20, 2026

Commits
LGTM

Review
Changes align v8 and v9 elasticsearch integration tests with the project pattern of ignoring context.Canceled in background stream.Run goroutines during expected shutdown.

LGTM

This was referenced Apr 20, 2026
@josephwoodward josephwoodward merged commit 1c8a23d into main May 4, 2026
7 checks passed
@josephwoodward josephwoodward deleted the mmt/elasticsearch-fixes-v2 branch May 4, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants